Implement Glean-based crash pings in desktop Firefox
Categories
(Toolkit :: Crash Reporting, task)
Tracking
()
Tracking | Status | |
---|---|---|
firefox111 | --- | fixed |
People
(Reporter: gsvelto, Assigned: afranchuk)
References
Details
Attachments
(1 file)
As per title, this would be the minimally viable ping as designed in bug 1790569 and would plug into the existing infrastructure, possibly here where we send the legacy ping.
Reporter | ||
Updated•2 years ago
|
Assignee | ||
Comment 1•2 years ago
|
||
Glean-based crash pings are sent alongside legacy crash pings.
Depends on D166925
Assignee | ||
Comment 2•2 years ago
|
||
As we are planning to support Firefox on Android as well, should the glean pings/metrics be added to the gecko_{pings,metrics}
categories rather than firefox_desktop_{pings,metrics}
? Or should we keep it as firefox_desktop
until we support Android too?
Comment 3•2 years ago
|
||
Chiming in to say: It's safe to do either of the options you propose. Putting them in gecko_*
will just create some empty tables until we start sending things from Fenix. Putting them in firefox_desktop_*
and then moving them later is supported by our infra.
Reporter | ||
Comment 4•2 years ago
|
||
Fenix is the next logical step so we can start with gecko_*
. The main driver behind this is having telemetry from Fenix too. History note: we used to have fully-featured crash pings back when Firefox for Android was Fennec, however they were lost during the transition to GeckoView.
Assignee | ||
Comment 5•2 years ago
|
||
That was my thought exactly, I've already made the change locally. I've just been waiting for data review to complete before sending the new patches.
Comment 7•2 years ago
|
||
bugherder |
Assignee | ||
Updated•2 years ago
|
Description
•